feat(agui): CopilotKit demo and HITL interrupt resume fixes - #2554
feat(agui): CopilotKit demo and HITL interrupt resume fixes#2554dragondyt wants to merge 2 commits into
Conversation
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
|
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
|
建议拆分一下，当前PR仅保留 CopilotKit 示例项目即可。Permission Interrupt 工具目前已经支持了，其它的可以单开小PR。 |
|
OK，我会基于最新的main分支拆分一下 |
deddde4 to
8f20214
Compare
Add a full-stack CopilotKit + AG-UI example and ThreadSessionManager support so demo/session APIs can verify threads, shared state, generative UI, and HITL flows end-to-end.
8f20214 to
6fba939
Compare
AgentScopeJavaBot
added
enhancement
AgentScopeJavaBot
left a comment
AgentScopeJavaBot
left a comment
There was a problem hiding this comment.
🤖 AI Review
This PR adds CopilotKit integration demo and HITL interrupt resume fixes to the AGUI module. The implementation includes a new CopilotKit protocol adapter, message type conversions, and WebSocket handler improvements. While the overall architecture is sound, there are several issues that need attention before merging.
Key concerns:
MessageContentItemis defined as a marker interface but used with type parameters that expect concrete methods, which could cause runtime type errors.- The WebSocket handler has a potential resource leak where sessions aren't properly cleaned up on connection close.
- The interrupt resume logic doesn't handle edge cases where the agent state is partially corrupted.
- Missing unit tests for critical conversion logic in
CopilotKitMessageConverter.
(inline comments could not be attached — line numbers fell outside PR hunks. See archived report.)
4 participants
Summary
agentscope-copilotkit) to verify threads, shared state, generative UI, A2UI workbench, and HITL flows end-to-end.ThreadSessionManagerso demo/session APIs can ensure named sessions and correctly resolve memory bythreadId.Depends on
MessageContent). Merge fix(agui): parse request bodies with Jackson 2 codec #2638 first, or keep this PR stacked on top of it.Related issues
Test plan
mvn spotless:checkon example / agui-spring-boot-starterThreadSessionManagerTestmvn -pl agentscope-examples/agentscope-copilotkit package(npm frontend build + Spring Boot repackage)agentscope-copilotkitbackend + frontend, confirm HITL interrupt/resume, thread session, shared state, and generative UI flows